Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.32%. Comparing base (343c078) to head (e11c73a).
⚠️ Report is 7 commits behind head on master.

@@            Coverage Diff            @@
##             master    #1302   +/-   ##
=========================================
  Coverage     56.31%   56.32%           
+ Complexity     8320     8319    -1     
=========================================
  Files           611      611           
  Lines         26726    26724    -2     
=========================================
+ Hits          15050    15051    +1     
+ Misses        11676    11673    -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I think it is useful to have a fallback using PartRepository::getPartBySPN with the lcsc part number.
That way the resolving is also useable if the part was not added via the LCSC provider

I think it is useful to have a fallback using PartRepository::getPartBySPN with the lcsc part number. That way the resolving is also useable if the part was not added via the LCSC provider

This has been added in.

The SPN resolval logic should also be part of the LCSC submethod (or we inline the complete method).

And I think most people will have the LCSC product code as SPN (as that is the order number) not with the MPN.